Skip to content

Use PANGAEA version of globcover and AWS for GHSL settlement - #32

Merged
sjpfenninger merged 4 commits into
mainfrom
fix/globcover-download
Nov 6, 2026
Merged

sjpfenninger merged 4 commits into
mainfrom
fix/globcover-download

Conversation

@irm-codebase

@irm-codebase irm-codebase commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Fixes #31 and increases download stability

Summary of changes in this pull request

  • Switched to the PANGEA version of GlobCover, which supports COG GeoTiff and uses a DOI.
  • GHSL now uses the official AWS version of the dataset (also COG).
  • Added both to the tiny_files option.
  • Added -f to curl requests so they fail clearly

I ran both the large and tiny file options and compared outputs against pre-existing results I had. Files are the same.

Reviewer checklist

  • There are no pip dependencies in the module's environment files (workflow/envs/).
  • All rules use pathvars (e.g., <results>) in their inputs and outputs.
  • The integration test-suite is successful, including:
    • pre-commit.ci tests pass.
    • tests pass for all relevant OS configurations (linux, osx, windows).
  • Module documentation is up-to-date, including:
    • INTERFACE.yaml mentions all relevant pathvars and wildcards.
    • README.md describes how to use the module and has the necessary citations.

@irm-codebase

Copy link
Copy Markdown
Member Author

@tud-mchen6, try using this version in your workflow. Should work well!

@irm-codebase irm-codebase changed the title Use PANGAEA version of globcover Use PANGAEA version of globcover and AWS for GHSL settlement Oct 27, 2026

@sjpfenninger sjpfenninger left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, many thanks!

@sjpfenninger
sjpfenninger merged commit dca25fe into main Nov 6, 2026
4 checks passed
@sjpfenninger
sjpfenninger deleted the fix/globcover-download branch November 6, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Download link for GlobCover broken

2 participants